testsuite: Fix popover-related failures

window: Make unrealize work again

GtkWidgets unrealize accesses the frame clock
of the surface, so we need to keep the surface
in place until after we've chained up.

Drop gtk_widget_set_surface

Not used anymore. Only GtkNative's have surfaces.

Drop GtkWidget::surface

Unneeded.

Work toward dropping widget->surface

Drop special-casing of GtkNative in
most widget vfuncs. GtkNative implementations
need to override these anyway.

Remove gtk_widget_get/set_has_surface

These serve no purpose anymore - widgets don't
have surfaces, unless they're a GtkNative.

Add a GdkSurface::parent property

popover: Bring back the beak

Bring back the code that draws an arrow.

x11: Fix handling of frame clock freezes

Now that popups share the frame clock of their
parent, we have to be much more careful about
freezing the clock, since that may stop updates
for another surface.

This commit makes two changes that make the
X11 handling of the frame clock more similar
to the Wayland backend:
- Use gdk_surface_freeze_updates instead of
  gdk_surface_freeze_toplevel_updates to avoid
  affecting the frame clock
- Bail out early in before_paint/after_paint
  if the surface is frozen, to avoid affecting
  the frame clock

Together, these two make the X11 popup surface
type work without freezing updates for the toplevel.

gdk: Inherit the frame clock for popups

With separate clocks, the phases are not coordinated,
which messes with GTKs size allocation machinery treating
the entire widget tree as a whole, and causes us to
run into assertion where popups get drawn before they
are allocated.

Add a testcase for size allocation

This triggers a critical warning in gtk_widget_snapshot.

widget: Remove a popover special case

We never run this since GtkPopover has its
own size_allocate, anyway.

text view: Allocate popovers

places sidebar: Allocate popovers

color swatch: Allocate popovers

entry: Allocate popovers

scale button: Allocate popovers

Call gtk_native_check_resize() from size_allocate,
as is required now. This gets volume buttons closer
to working again (dragging the slider still doesn't
work).

text: Allocate popovers

We need to do the same thing the menu button
does, and call gtk_native_check_resize for
all our popovers

window: Don't inherit cursors across surfaces

I can't think of a case where this is the desired
behavior. So, instead of setting an explicit cursor
on all popups, just stop walking the parents at
surface boundaries.

widget: Optimize resize propagation

When a GtkNative widget is marked as resize_needed,
we need a current position for its parent and we need
the parent to be allocated (so we can position our
surface), but we don't need the parent to be marked
as resize_needed, since the parent size is entirely
independent of the popup size.

widget: Improve warning messages

When we print warnings about a widget, using
gtk_widget_get_name() is slightly better than
G_OBJECT_TYPE_NAME(), since it will give us
the widgets unique name when available.

Add gdk_surface_get_parent back

The api is the same, the parent is different.
This now returns the parent of popup surfaces.

surface: Remove gdk_surface_set/merge_child_input_shapes()

These are useless now that we don't have child surfaces

surface: Remove old debug code

popover: Use GdkSurface::autohide

The new api is made for this case.

surface: Add an autohide property

This api is meant to mimic xdg-popover.grab - we
show the surface, and dismiss it when we get events
on other surfaces. For foreign surfaces, the compositor
handles that for us; for our own, we check outselves
before delivering events to GTK.

broadway: Track popups

This is more or less a 1:1 copy of the X11 code.

x11: Implement popup surfaces

Make them use o-r windows, and move
with their parent.

We do a sort-of ok job on stacking order
here - whenever the parent window gets a
ConfigureNotify, we just restack all popups
directly on top of their parent. This is good
enough to keep popups on top of their parent
while we drag it around, and it gets the popup
to disappear when raising another window on
top of the parent.

wayland: Use popup parents

Make the Wayland move-to-rect implementation
use popup parents.

surface: Stop setting transient-for for popups

This was just a hack to reuse the existing
infrastructure. Now that we store parents separately,
stop doing it.

surface: Make move-to-rect work for popups

Now that we store popup parents separately,
the client-side move-to-rect implementation
must use them.

surface: Store popup parent

Store popup parents separately from transient-for
parents, since these are separate concepts with
different behaviors. And we need the parent in
the frontend, so we can use it in the fallback
move-to-rect implementation.

gdk: Introduce a popup surface type

This surface type is meant to have semantics
similar to xdg-popup, with a mandatory parent,
and relative placement using gdk_surface_move_to_rect.

x11: Remove useless checks

All surfaces are toplevels now, no need
to check for this all over the place.

Get rid of GdkSurfaceImpl

We don't need the complicated wrapper system anymore,
since client-side windows are gone. This commit moves
all the vfuncs to GtkSurfaceClass, and changes the
backends to just derive their surface implementation
from GdkSurface.

gdk: Add a private header

This is in preparation for getting rid of GdkSurfaceImpl.

Rename surface constructors

We want to use a gdk_surface_new_popup for popups,
and align the constructor names with the surface
types, so rename

gdk_surface_new_popup -> gdk_surface_new_temp
gdk_surface_new_popup_full -> gdk_surface_new_popup

The temp surface type will disappear eventually.

surface: Cosmetics

gdk: Drop pass-through surfaces

With client-side windows gone, this property
is no longer used at all, and has been replaced
by a similar concept on the widget level.
Drop it.

gdk: Drop GdkWindowAttr

All the information in it is already contained
in the surface object we pass along, and none
of the backend implementations were using the
attributes at all.

gdk: Drop input-only surfaces

We are not creating such surfaces anymore, and
they were only ever meaningfully implemented
on X11. Drop the concept, and the api for determining
if a surface is input-only.

gdk: Remove an unused field

Spring cleaning. We are not using this field
at all, since we have the input_only boolean.

Drop gdk_surface_new_temp

This is no longer used, and the only difference
to gdk_surface_new_popup is input-only, which is
an X11-only concept.

menu: Stop using gdk_surface_new_temp

We can just as well use gdk_surface_new_popup.

x11: Stop using gdk_surface_new_temp

We can just as well use gdk_surface_new_popup.

broadway: Stop using gdk_surface_new_temp

This is in preparation for adding a proper
popup surface type.

wayland: Remove a useless switch

We are setting the title unconditionally, now

docs: Remove references to gdk_device_grab

It is no longer public api, refer to gdk_seat_grab instead.

gdk: Remove remnants of client-side windows

Now that all surfaces are native, we don't need
to separately keep track of grab surfaces and
their native ancestor anymore.

Drop gtk_widget_register_surface

This is no longer used.

popover: Stop using gtk_widget_register_surface

The GtkRoot implementations are expected to handle
the necessary surface setup themselves, going forward.

window: Stop using gtk_widget_register_surface

The GtkRoot implementations are expected to handle
the necessary surface setup themselves, going forward.

menu: Stop using gtk_widget_register_surface

It does not seem to be necessary at all.

surface: Make gdk_surface_get_device_position void

None of the callers were using the return value,
and without child surfaces, it is not very useful.

x11: Drop a gratitious use of gdk_event_get_root_coords

We are in the backend, so we can do the translation ourselves.

Remove gdk_surface_get_geometry as public api

We still need to keep the vfunc around, since the
fallback implementation of gdk_display_get_monitor_at_surface
uses it. So, a GDK backend must either have root coordinates
or always return a monitor from monitor_at_surface.

Drop gdk_surface_get_root_origin

It was not used.

Remove gdk_surface_get_root_coords as public api

We still need to keep the vfunc around, since the
fallback implementation for move_to_rect uses it.
So, a GDK backend must either have root coordinates
or implement move_to_rect.

Remove gdk_surface_coords_to/from_parent

This api was unused, and surfaces have no parents anymore.

surface: Small simplification

No more special-casing of state setting.

surface: Simplify destroy implementation

Without child surfaces, there is no need to recurse.

surface: Remove abs_x/abs_y

We no longer have child windows, so the offset is
always 0.

surface: Small cleanup

Every surface has an impl now, no need to check.

surface: Simplify some code

Fold gdk_surface_process_updates_recurse into its
only caller. There is no recursion here anymore.

Remove surface->children

This field is not used at all anymore.

gdk: Stop using surface->children

It is always NULL.

wayland: Stop using surface->children

It is always NULL.

Remove surface->parent

This field is no longer used.

gdk: Stop using surface->parent

It is always NULL.

broadway: Stop using surface->parent

x11: Stop using surface->parent

It is always NULL.

wayland: Stop using surface->parent

It is always NULL.

surface: Remove code dealing with child surfaces

We no longer have child surfaces, so this code
is never run.

surface: minor cleanup

Make gdk_surface_new fully private, and reduce the use
of GdkSurfaceAttr.

Drop child surfaces

Drop gdk_surface_child_new and the child surface type,
since we no longer use them. Deprecate surface apis
that only make sense with child surfaces.

widget: Stop using child surface apis

All our surfaces are toplevels now, and thus all native.

a11y: Stop supporting ATK_XY_SCREEN

We were already not supporting ATK_XY_PARENT, and we can't
support global positions, so just always do ATK_XY_WINDOW.

xim: Stop using child surface apis

We no longer have child surfaces.
Drop code that only makes sense in
their presence.

wayland: Stop using child surface apis

We no longer have child surfaces.
Drop code that only makes sense in their
presence.

x11: Stop using child surface api

We no longer have child surfaces.
Drop code that only makes sense in
that case.

entry completion: Use a popover

This lets us remove a use of GTK_WINDOW_POPUP,
which should eventually be going away.

We need to disable treeview search, since it
creates a toplevel that will disrupt our grabbing
popup, causing it to be dismissed.

We don't need to grab ourselves, since the popover
code does it for us. We don't need to reposition our
window, since the popover takes care of that too.

main: Use gtk_widget_get_native

main: Remove special-case hiding popopvers

This is now happening in GDK, so we don't have
to do this here anymore.

widget: Remove some unneeded popover special-casing

Adwaita: Give popover menus a background

This makes the hover highlight on menu items visible.

menubutton: Allocate the popover

This is temporary, until we've figured out
the proper way of hooking this up

Adapt to new popover lifecycle

We need to unparent popovers in dispose.

Reimplement GtkPopover

surface: Start sketching a new surface type

Start by adding a constructor. We have to call it
gdk_surface_new_popup_full for now, since gdk_surface_new_popup
is taken. This may be reshuffled later.

menushell: Keep outside clicks working

We need to unset event propagation limits here
to keep receiving outside clicks that cause us
to close the menu.

Introduce event controller propagation limits

Limit event handlers by default to only handle
events targeting the same surface as their widget.

widget: Don't snapshot foreign children

When snapshotting, we walk down the widget tree.
We need to skip children that have a different
surface, since those will do their own snapshot.

root: Make gtk_root_get_display public

This is following the precedent of making
GtkNative getters public.

GtkRoot: Drop overlap with GtkNative

Drop the parts from the GtkRoot interface
that have been taken over by GtkNative.

widget: Use GtkNative

Review and replace uses of get_toplevel

Now that roots can have parent widgets, we need to
carefully examine all calls of gtk_widget_get_toplevel,
and replace them with gtk_widget_get_root if we want
the nearest root, and not the ultimate end of the parent
chain.

Use GtkNative APIs instead of GtkRoot

Add gtk_widget_get_native

This is a common enough operation to deserve api.

root: Require GtkNative

window: Implement GtkNative

Adapt GtkWindow to implement both GtkRoot and GtkNative.